Skip to content

fix: update Vercel deployment instructions to remove incorrect Output…#1413

Open
sudha09-git wants to merge 1 commit into
magic-peach:mainfrom
sudha09-git:fix/1238-deployment-fixes
Open

fix: update Vercel deployment instructions to remove incorrect Output…#1413
sudha09-git wants to merge 1 commit into
magic-peach:mainfrom
sudha09-git:fix/1238-deployment-fixes

Conversation

@sudha09-git
Copy link
Copy Markdown
Contributor

Summary

Fixes incorrect Vercel deployment instructions in README.md that caused routes-manifest.json couldn't be found errors.
Closes #1238


Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup

Changes Made

  • Updated both Vercel deployment sections in README.md
  • Removed hardcoded out as Output Directory
  • Added note that Vercel auto-detects the output directory for Next.js static exports

How to Test

  1. Follow the updated README deployment instructions
  2. Deploy to Vercel without setting Output Directory manually
  3. Vercel should auto-detect out and deploy successfully

Checklist

  • Linked issue in summary
  • Self-reviewed the diff
  • No TypeScript errors

@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

@sudha09-git is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @sudha09-git!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

What happens next

  1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
  2. Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
  3. 👀 Code review — a maintainer will review your changes
  4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

  • PR title follows Conventional Commits (e.g. feat: add dark mode)
  • Linked the issue this PR closes (e.g. Closes #123)
  • Tested the changes locally (bun run dev)
  • Build passes (bun run build)

Useful links

Happy coding! 🎉

@github-actions github-actions Bot added level:beginner Beginner level - 20 pts type:bug Bug fix type:docs Documentation type:feature New feature type:refactor Code refactor type:testing Testing labels May 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

✅ PR Format Check Passed — @sudha09-git

Basic format checks passed. A maintainer will review your code changes.

This does not mean the PR is approved — it just means the format is correct.

@sudha09-git
Copy link
Copy Markdown
Contributor Author

Hi @SachinSrivastava777 and @magic-peach 👋

I've been assigned to this issue and have submitted a fix in PR #[1413].

Changes made:

  • Updated both Vercel deployment sections in README.md to remove the hardcoded Output Directory: out setting
  • Added a note that Vercel auto-detects the out directory for Next.js static exports — setting it manually causes the routes-manifest.json couldn't be found error

Note on the duplicate React import: After investigation, ThemeToggle.tsx no longer has a duplicate import in the current codebase. The only import React found is in ErrorBoundary.tsx, which is required since it uses a class component extending React.Component.

Please review and let me know if any changes are needed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:beginner Beginner level - 20 pts type:bug Bug fix type:docs Documentation type:feature New feature type:refactor Code refactor type:testing Testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Deployment Fixes

1 participant